-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
OL3 style changes for controls (esp. zoom) #56
Conversation
Conflicting PR. Removed from build iviewer-merge#244. See the console output for more details.
|
Looking good.
Will give you this: |
@will-moore I have included the css changes mentioned and altered the scalbar style. There is a 1px margin on .class ol-scale-line-inner still. I think without it contrast is quite low but have a go yourself and remove it to see the results. |
Other css changes look good. |
the adjusted scale bar looks nice easier to find an allows to check distance between element |
Looks good, thanks. |
see: ome/design#75
If there are any requests for style changes, please be as specific as possible. The css colors and styles I cannot really fully deduce from looking at the mockups only. So if the there is a preferred style and it is already in the css just point me toward it, if not just give me the color in any notation or the respective css property that should be applied. That way we avoid a guessing and back and forth.